-
Notifications
You must be signed in to change notification settings - Fork 6.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[poco] Fix mingw compilation #42330
[poco] Fix mingw compilation #42330
Conversation
for test I created a project like below. I think it will be useful for reviewers. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMO you should only add the patch.
Then increment port-version.
Then commit, then update the versions registry.
I couldn't find any comprehensive resources on how to manage |
f606d7c
to
fab6121
Compare
Thanks for the fix! |
Fixes #38873
Fixes #39797
The development has been tested with
x64-mingw-static
andx86-mingw-static
and no issues have been observed.Due to the more patches in this port, I have scoped the patch specifically to this triplet, as it seemed more appropriate. This approach should not cause any issues for other triplets.